Skip to content

Fix/replace prettier with biome#330

Open
Subham-KRLX wants to merge 6 commits intoNextCommunity:mainfrom
Subham-KRLX:fix/replace-prettier-with-biome
Open

Fix/replace prettier with biome#330
Subham-KRLX wants to merge 6 commits intoNextCommunity:mainfrom
Subham-KRLX:fix/replace-prettier-with-biome

Conversation

@Subham-KRLX
Copy link
Contributor

Fixes #318

Replaced the archived Prettier pre-commit hook with Biome for faster linting and formatting of JS, CSS, and JSON files. This PR updates
.pre-commit-config.yaml
and
biome.json
, and includes automated formatting fixes for existing scripts and config files.

Verified locally with pre-commit run --all-files.

@Subham-KRLX Subham-KRLX requested a review from jbampton as a code owner January 29, 2026 11:39
- Removed archived mirrors-prettier hook
- Enhanced biome-check hook to handle JS, CSS, and JSON
- Updated biome.json to handle all supported files
- Fixed typo in .pre-commit-config.yaml (minimum_pre_commit_version)
- Applied automated formatting fixes via Biome
@Subham-KRLX Subham-KRLX force-pushed the fix/replace-prettier-with-biome branch from f9efb24 to 6392db7 Compare January 29, 2026 11:43
@Subham-KRLX
Copy link
Contributor Author

I have resolved the merge conflicts and fixed all the Biome linting errors (unused variables, imports, and CSS priorities).

@jbampton
Copy link
Member

jbampton commented Feb 3, 2026

Are we able to do everything but the CSS ?

@Subham-KRLX
Copy link
Contributor Author

I have updated the configuration to exclude CSS files from Biome and reverted the changes to style.css. Biome is now set to run only on JS and JSON files.

@jbampton
Copy link
Member

jbampton commented Feb 5, 2026

I have updated the configuration to exclude CSS files from Biome and reverted the changes to style.css. Biome is now set to run only on JS and JSON files.

https://discord.gg/sQrjUYA6zd

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need this CSS ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes it's required This file contains custom theming the Developer Tools panel Matrix effects and user card styling that Tailwind doesn't cover. It's linked in index.njk and bio.njk I have reverted it to match main in this PR.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

what about the deleted lines from 915 onwards ?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have properly reverted the file to match main. The deleted lines have been restored. The file is now identical to main with no changes in this PR.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jbampton jbampton requested a review from BaseMax February 6, 2026 07:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: In progress

Development

Successfully merging this pull request may close these issues.

Remove prettier hook and replace with Biome

2 participants